Skip to content

Conversation

@svara
Copy link
Contributor

@svara svara commented Jan 14, 2026

This PR exposes pins API endpoints (pin/unpin and pinned cards list).

Pin
POST /:account_slug/cards/:card_number/pin
Returns: 204 No Content

Unpin
DELETE /:account_slug/cards/:card_number/pin
Returns: 204 No Content

Pinned cards list
GET /my/pins
Returns: an array of card objects (same shape as other card JSON responses).

@svara svara requested a review from jayohms January 15, 2026 16:38
@svara svara marked this pull request as ready for review January 15, 2026 16:38
Copy link
Contributor

@jayohms jayohms left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, thanks!

Copy link
Collaborator

@kevinmcconnell kevinmcconnell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great!

Copy link
Collaborator

@kevinmcconnell kevinmcconnell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, @svara!

svara and others added 2 commits January 27, 2026 15:37
* main: (127 commits)
  Align board name start so it doesn't get too big
  Clean up card/events header layout
  Sync email to Stripe when user changes email address (#2432)
  Fix IDOR in webhook activation endpoint (#2431)
  Add card reactions to API docs and reactions_url to card JSON (#2427)
  Remove unnecessary claude plan
  Allow boosts on cards (#2411)
  Revert "Fix notification click URL by using correct data property"
  Add migration to remove draft cards from search index
  Guard search indexing with searchable? check
  Forbid comments on draft cards
  prefactor: update search to use published cards
  Fix notification click URL by using correct data property
  Wait for service worker to be active before subscribing
  Fix stuck state when permission granted but no subscription
  Extract Card::Commentable
  Include arm64 build in Docker workflow
  Remove unnecessary `await` in push handler
  Correctly initialise WebPush connection (#2417)
  Update models, views, and fixtures for polymorphic reactions
  ...
@svara svara merged commit ec1b8f2 into main Jan 27, 2026
12 checks passed
@svara svara deleted the pins-api branch January 27, 2026 14:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants